Skip to content

shared-config: deprecate zizmor.yml#452

Merged
MikeMcQuaid merged 1 commit into
mainfrom
deprecate-zizmor-yml
Jul 11, 2026
Merged

shared-config: deprecate zizmor.yml#452
MikeMcQuaid merged 1 commit into
mainfrom
deprecate-zizmor-yml

Conversation

@p-linnane

Copy link
Copy Markdown
Member

Now that Homebrew/actions usages are SHA-pinned to versioned releases, the .github/zizmor.yml exception allowing Homebrew/actions/* ref pins is obsolete. Remove it and mark it deprecated in the sync so existing downstream copies are cleaned up on the next cycle.

Also drop the older lock-threads.yml and stale-issues.yml cleanups, which have already propagated through every repository in the sync matrix.

Signed-off-by: Patrick Linnane <patrick@linnane.io>
Copilot AI review requested due to automatic review settings July 10, 2026 20:48

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the now-obsolete .github/zizmor.yml exception for Homebrew/actions/* ref-pin usage and updates the shared-config sync logic to treat that file as deprecated so downstream repositories will have it removed on the next sync cycle.

Changes:

  • Delete .github/zizmor.yml from the shared config.
  • Update the shared-config sync script to remove .github/zizmor.yml from target repositories (deprecation cleanup).
  • Remove no-longer-needed deprecation cleanup handling for older workflow files that have already fully propagated.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/zizmor.yml Removes the obsolete zizmor rule exception now that Homebrew/actions usages are release/SHA pinned.
.github/actions/sync/shared-config.rb Updates sync behavior to delete downstream .github/zizmor.yml copies as a deprecated file and removes older deprecated workflow cleanup paths.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MikeMcQuaid MikeMcQuaid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@MikeMcQuaid MikeMcQuaid merged commit dc80003 into main Jul 11, 2026
27 checks passed
@MikeMcQuaid MikeMcQuaid deleted the deprecate-zizmor-yml branch July 11, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants